GTK: Fix builds against latest stable GLib with MSVC

GLib-2.68.x now considers warning C4098 ('void' function returning a value) as
an error, so avoid doing that.

GSK: Fix builds against latest stable GLib with MSVC

GLib-2.68.x now considers warning C4098 ('void' function returning a value) as
an error, so avoid doing that.

GDK: Fix builds against latest stable GLib with MSVC

GLib-2.68.x now considers warning C4098 ('void' function returning a value) as
an error, so avoid doing that.

Merge branch 'im-context-event-leak' into 'master'

imcontext: Eat key events during preedit

See merge request GNOME/gtk!3440

Merge branch 'theme-fallback-fix' into 'master'

themes: Make fallback work a bit better

See merge request GNOME/gtk!3441

themes: Make fallback work a bit better

One case we did not include in fd329e0ec582d7562f444b is

Adwaita-dark -> Default:dark

It turns out that it is needed.

imcontext: Eat key events during preedit

Avoid passing through random key press or release
events while we are showing preedit. That prevents
'accidents' like typing Ctrl-. bringing up the
Emoji chooser during preedit, or hitting Ctrl-a
after the Compose key moving the 'dot' around in
vim in terminals.

Merge branch 'demo-manpages' into 'master'

meson: only build demo manpages when demos are enabled

See merge request GNOME/gtk!3438

Merge branch 'theme-icon-filter-fixes' into 'master'

theme: Use -gtk-icon-filter for full-color icons

See merge request GNOME/gtk!1990

theme: Use -gtk-icon-filter for full-color icons

Not for symbolic icons.

Don't apply `-gtk-icon-filter: opacity(0.5);` to the symbolic icons as
they already have the "gray" colors indicating the disabled state.
Symbolic icons can be styled using the `color` property.

Also remove the obsolete comment.

meson: only build demo manpages when demos are enabled

Installing them even when manpages are enabled, but the demos are
disabled, produces manpages for unavailable programs...

Merge branch 'x11-quiet-exit' into 'master'

x11: Be quiet on exit by default

See merge request GNOME/gtk!3436

Merge branch 'ngl-crash' into 'master'

ngl: Fix a rare assertion violation

Closes #3853

See merge request GNOME/gtk!3434

x11: Be quiet on exit by default

The condition we check for to catch X servers going away
may not be accurate anymore, and the warning shows up in
logs, causing customers to be concerned. So, be quiet by
default, unless the user explicitly asked for a message.

Merge branch 'flatpak-highlight' into 'master'

flatpak: Include highlight in gtk4-demo flatpak

Closes #3867

See merge request GNOME/gtk!3433

flatpak: Include highlight in gtk4-demo flatpak

Include the highlight utility to make syntax
highlighting work in gtk4-demo. This requires
building its dependencies as well, boost and lua.

ngl: Fix a rare assertion violation

When we clean up the uniform allocations after a frame,
it can happen that our space requirements actually increase,
due to padding that depends on the order of allocations.

Instead of asserting that it doesn't happen, just make
it work by growing our allocation.

Fixes: #3853

Merge branch 'testsuite-asserts' into 'master'

testsuite: Reduce the use of g_assert

Closes #3864

See merge request GNOME/gtk!3432

build: Quiet the compiler

Don't use -Wnull-dereference when assertions are
disabled, since that causes the compiler to see
NULL everywhere.

wayland: Build with G_DISABLE_ASSERT

build: Don't append multiple flags as a string

The gtk_debug_cflags variable is an array, with each argument stored in
a separate string; if we add multiple arguments inside the same string
we break the escaping rules.

Fixes: #3864

gsk/ngl: Build with G_DISABLE_ASSERT enabled

Force g_test_init to cooperate

Change all g_test_init calls in the testsuite
to avoid the macro, so they work regardless of
G_DISABLE_ASSERT.

Make gtk_test_init work without assertions

The g_test_init macro 'helpfully' refuses to work when
assertions are disabled. So we just use the function.

Remove assertions in tests

Inspector: Fix a possible undefined use

Fix unused variable warnings

These happen for variables which are only used
in assertions, when assertions are disabled.

gtk-demo: Avoid compiler warnings without assertions

Avoid an unused variable warning.

gsk: Avoid compiler warnings without assertions

Disabling assertions was provoking some unused variable
warnings from the compiler. Avoid these.

testsuite: Reduce the use of g_assert

Stop using g_assert() in the gtk tests.

testsuite: Reduce the use of g_assert

Stop using g_assert() in the gsk tests.

Merge branch 'f16c.msvc' into 'master'

Fix F16C build/runtime detection on Visual Studio

See merge request GNOME/gtk!3429

Merge branch 'testsuite-asserts' into 'master'

testsuite: Reduce use of g_assert

See merge request GNOME/gtk!3431

testsuite: Reduce use of g_assert

Stop using g_assert() in the css tests.

testsuite: Reduce use of g_assert

Stop using g_assert() in the reftests.

testsuite: Reduce use of g_assert

Stop using g_assert() in the gdk tests.

meson.build: Check for F16C on 32-bit Visual Studio

It is also possible to enable F16C instructions on 32-bit Visual Studio builds,
so also check for the compiler's ability to build F16C code on Visual Studio
32-bit instead of just bailing out.

gsk/ngl/fp16.c: Implement runtime F16C detection on MSVC

We need to use __cpuid() to check for the presence of F16C instructions on
Visual Studio builds, and call the half_to_float4() or float_to_half4()
implementation accordingly, as the __builtin_cpu...() functions are strictly
for GCC or CLang only.

Also, since __m128i_u is not a standard intrisics type across the board, just
use __m128i on Visual Studio as it is safe to do so there for use for
_mm_loadl_epi64().

Like running on Darwin, we cannot use the alias __attribute__ as __attribute__
is also for GCC and CLang only.

meson.build: Improve F16C detection on Visual Studio

The __builtin_cpu...() intrisics are strictly for GCC/CLang-based compilers, so
don't use them in the checking code on Visual Studio.  The test code will still
compile without this change, but will certainly fail if we want it to link.

Merge branch 'ngl-shadow-fixes' into 'master'

ngl: Fix outset blurred shadows

See merge request GNOME/gtk!3427

Merge branch 'grouped-toggle-fix' into 'master'

togglebutton: Fix expected group behavior

See merge request GNOME/gtk!3425

Merge branch 'shadow-reduction' into 'master'

gsk: Don't overshadow

See merge request GNOME/gtk!3419

ngl: Fix unevenly scaled shadows

Ensure that we don't cut them off at the edges.

ngl: Don't slice unsliceable shadows

When the corners are too big to make slicing work,
don't do it, since it leads to broken results.

ngl: Fix a case of flipped shadow

In the non-sliced case, we were rendering the shadow
upside down.

ngl: Add code to dump shadows

This can be helpful in understanding shadow bugs.

Merge branch 'matthiasc/for-master' into 'master'

menubutton: Propagate focus-on-click

See merge request GNOME/gtk!3424

togglebutton: Fix expected group behavior

The expectation is that a grouped button stays
active when you click it. Grouped check buttons
behave this way, and grouped toggle buttons should
too.

node-editor: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

icon browser: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

widget-factory: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

gtk-demo: Make header buttons not steal focus

This is rarely useful, and is confusing when it
happens as side-effect of dragging the window.

menubutton: Propagate focus-on-click

Propagate the focus-on-click setting to the button
inside, so that setting menubuttons as !focus-on-click
works as expected. This helps for menubuttons in
header bars, where dragging on the button will otherwise
steal focus from the content.

Update Ukrainian translation

Merge branch 'matthiasc/for-master' into 'master'

some small optimizations

See merge request GNOME/gtk!3423

ngl: Allow passing bounds to update_clip

update_clip only uses the bounds of the node that
we are currently passing. This opens the door to
updating the clip while handling a single node.

ngl: Small optimization for shadows

When 9-slicing shadows, omit the center tile when it is
entirely contained in the outline (that is not always
the case, depending on corners and offsets).

roundedrect: Speed up contains_rect and friends

gsk_rounded_rect_contains_rect was calling
gsk_rounded_rect_contains_point, which potentially
checks all four corners, for a total of up to 16
corner/point checks. But there is no need to do
more than 4 such checks to answer the question.

ngl: Small optimization

We really want equality here.

nodeeditor: Make the help window wide enough

Make the help window wide enough to show the
tables without wrapping.

Merge branch 'media-memory-leak' into 'master'

media: Fix a memory leak

See merge request GNOME/gtk!3420

media: Fix a memory leak

This was introduced in e7dc82fa32f9c115607.

gsk: Don't overshadow

Reduce the cairo shadows to the same size as their GL brethren.

Merge branch 'popover-list-lockup' into 'master'

window: Defer focus setting until after paint

Closes #3725

See merge request GNOME/gtk!3417

window: Defer focus setting until after paint

Commit 3dbf5038fab8eb0d8c11 tried to defer focus changes
until after rendering is done. But it failed to do so, since
the toplevel ::render handler is still before rendering of
popups that are attached to that toplevel. To do this
properly, we need to do it in the AFTER_PAINT frame clock
phase.

Fixes: #3725

Merge branch 'ngl-coloring-underlines' into 'master'

Optimize underlines in text

See merge request GNOME/gtk!3414

Merge branch 'no-scroll-cursor' into 'master'

scrolledwindow: Stop using scroll cursors

See merge request GNOME/gtk!3416

Merge branch 'ebassi/ci-docs' into 'master'

ci: Drop the split Fedora images

See merge request GNOME/gtk!3415

scrolledwindow: Stop using scroll cursors

We used to override cursor to use all-scroll while the
content is being scrolled. Unfortunately, there is several
problems with this:
- It is really only expected certain devices, and we don't
  have the device information on Wayland
- With the way cursor setting works in GTK4, non-NULL cursors
  of the content (eg the text views ibeam) win, making the
  scroll cursor not show up
- Under X11, we seem to miss scroll end events and then
  the scroll cursor gets stuck
Therefore, just remove this feature.

ci: Drop the split Fedora images

Now that we don't need the whole of Pandoc to build our documentation,
we can go back to a single Fedora image for our CI.

ngl: Add a program change counter

Count how often we change programs during a frame.

ngl: Optimize underlines in text

Opportunistically use the coloring program for
drawing underlines instead of the color program.
This avoids program changes in the middle of
text.

For the Emoji text scrolling benchmark, this reduces
the program changes per frame from > 1000 to around 100.

ngl: Reserve a pixel in texture atlases

This will be used for coloring from the texture.

ngl: Fix an oversight

All the rest of debug spew goes to stderr here.

gtk-demo: Make one text scroll benchmark harder

Add underlines into the mix with the Emoji heavy
text, to see if we can optimize lines.

Merge branch 'half-float' into 'master'

ngl: Use fp16 for colors

See merge request GNOME/gtk!3405

Merge branch 'font-fallback-fix' into 'master'

wayland: Improve font setting fallback more

Closes #3838

See merge request GNOME/gtk!3412

Work around compiler shortcomings on macOS

alias attributes don't work on Darwin, so
do without.

gsk: Add runtime checks for F16C

Use an IFUNC resolver to determine whether we can use
intrinsics for FP16 conversion. This requires the functions
to be no longer inline.

Sadly, it turns out that __builtin_cpu_supports ("f16c")
doesn't compile on the systems where we want it to prevent
us from getting a SIGILL at runtime.

Add an option to disable F16C

And add a compile time check for the presence of this
CPU feature.

Move color conversion up

Doing the color conversion early lets us reuse
the results whenever we do more than one draw
per node.

ngl: Use a constant for (fp16)-1

No need to convert that again for every glyph.

ngl: Reduce vertex storage

We only have one shader that uses the color2 attribute,
and it doesn't use the uv attribute, so save vertex
memory by putting those in the same space.

This reduce the per vertex space from 32 to 24 bytes.

ngl: Use fp16 for colors

This reduces the size of our Vertex struct from
48 to 32 bytes. It would be nicer if we could store
the colors in fp16 format in the rendernodes, and
avoid conversion here. But this is still good.

wayland: Improve font setting fallback more

We may get a response from the portal that contains
no useful settings at all. In that case, we should
fallback as well.

Fixes: #3838

Merge branch 'wip/carlosg/stuck-frames' into 'master'

surface: Always set PHASE_PAINT as pending when updates are scheduled

Closes #3750

See merge request GNOME/gtk!3411

surface: Always set PHASE_PAINT as pending when updates are scheduled

At times (most often when closing subsurfaces that are scheduling
relayouts) the PHASE_PAINT handling gets broken with the following
sequence:

1. Surface receives wl_callback.done for the previous frame.
   Surface is thawed.
2. A new update on the surface is scheduled. PHASE_PAINT is
   requested directly on the frame clock. priv->pending_phase is
   left unset in the surface.
3. Surface gets frozen
4. Frame clock processes the update scheduled at 2. The surface
   is frozen, so paint is prevented. PHASE_PAINT is considered
   handled.
5. Compositor emits wl_callback.done again. Surface is thawed.
6. At this point the machinery is off
   - The surface didn't paint but has pending update regions
   - priv->draw_needed is set in the toplevel and other portions
     of the widget tree
   - So queueing redraws is ineffective at eventually calling
     gdk_surface_schedule_update() again on the toplevel surface.
   - We don't paint anymore, so this broken state is not flushed
     until other subsurface changes manage to schedule the missing
     update.

To fix this, always set PHASE_PAINT in priv->pending_phase when
doing gdk_surface_schedule_update(). If the frame clock turns
around before the surface is thawed, it will still be waiting to
be processed the next iteration.

Fixes: https://gitlab.gnome.org/GNOME/gtk/-/issues/3750

Merge branch 'font-settings-fallback-4' into 'master'

wayland: Improve font setting fallback

See merge request GNOME/gtk!3407

wayland: Improve font setting fallback

When we don't get stettings from the portal, the current
fallback is 'awful fonts'. There is no need for that. Instead,
set the fallback values to grayscale antialiasing with slight
hinting.

Merge branch 'textview-last-line' into 'master'

textview: Fix a problem with the last line

Closes #3835

See merge request GNOME/gtk!3403

textview: Fix a problem with the last line

Sometimes, we missed the last line for display.

Fixes: #3835

Merge branch 'matthiasc/for-master' into 'master'

window: Actually enable the inspector by default

Closes #3831

See merge request GNOME/gtk!3401

modelbutton: Avoid a crash

We may not have a popover menu ancestor. The check for
this was forgotten in one of the branches here.

Fixes: #3831

window: Actually enable the inspector by default

427d216081ea0fd9e2045e changed the default in the schema,
but didn't handle cases where the schema isn't found.

Merge branch 'matthiasc/for-master' into 'master'

textlayout: Remove some unnecessary code

See merge request GNOME/gtk!3400

ngl: Fix an oversight

We were special-casing 2D affine transforms,
but overlooked even simpler transforms.

textlayout: Remove an unused function

textlayout: Avoid allocations for line iteration

No need to collect the lines firs, and allocate memory
for that. We can just iterate over them right away.

textlayout: Introduce some locals

No need to call _gtk_text_buffer_get_btree over and over.

textlayout: Create the cursor snapshot on demand

We don't need it all the time.